Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

MIMXRT1062 IVT needs to be FIRST #79611

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

MIMXRT1062 IVT needs to be FIRST #79611

wants to merge 1 commit into from
+1 −0

Conversation

RobinKastberg
Copy link

IMXRT1062 bootrom reads boothdr initial vector table from 0x60001000. In the CMAKE scatter linker scripts we put multiple sections at offset 0x1000 in the rom. In linkers other than LD, we are not guaranteed a particular order when placing these. If we specify FIRST we can count on the .ivt coming first. The other positions aren't as crucial.

From IMXRT1060RM.pdf 9.7.1

The location of the IVT is the only fixed requirement by the ROM.
The remainder or the image memory map is flexible and
is determined by the contents of the IVT.

@RobinKastberg
MIMXRT1062 IVT needs to be FIRST
be43ad7 
IMXRT1062 bootrom reads boothdr initial vector table
from 0x60001000. In the CMAKE scatter linker scripts we put multiple
sections at offset 0x1000 in the rom. In linkers other than LD, we are
not guaranteed a particular order when placing these.
If we specify FIRST we can count on the .ivt coming first. The other
positions aren't as crucial.

From IMXRT1060RM.pdf 9.7.1

> The location of the IVT is the only fixed requirement by the ROM.
> The remainder or the image memory map is flexible and
> is determined by the contents of the IVT.

Signed-off-by: Robin Kastberg <robin.kastberg@iar.com>
@zephyrbot zephyrbot added size: XS A PR changing only a single line of code platform: NXP NXP labels Oct 9, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 9, 2024

Hello @RobinKastberg, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@decsny
Copy link
Member

decsny commented Oct 9, 2024

which other sections are being placed at 0x1000?

@RobinKastberg
Copy link
Author

@decsny
It is in the same statement that I modified. It's these

          ".boot_hdr.ivt"
          ".boot_hdr.data"
          ${boot_hdr_dcd_data_section}
          ${boot_hdr_xmcd_data_section}

@dleach02
Copy link
Member

dleach02 commented Oct 9, 2024

@hakehuang could you run some regression on the imxrt platforms with this PR?

@hakehuang
Copy link
Collaborator

@hakehuang could you run some regression on the imxrt platforms with this PR?

regression in progress, will feedback once done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@butok butok Awaiting requested review from butok

@danieldegrasse danieldegrasse Awaiting requested review from danieldegrasse

@decsny decsny Awaiting requested review from decsny

@DerekSnell DerekSnell Awaiting requested review from DerekSnell

@dleach02 dleach02 Awaiting requested review from dleach02

@EmilioCBen EmilioCBen Awaiting requested review from EmilioCBen

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@yvanderv yvanderv Awaiting requested review from yvanderv

At least 2 approving reviews are required to merge this pull request.

Assignees

@mmahadevan108 mmahadevan108

@dleach02 dleach02

Labels
platform: NXP NXP size: XS A PR changing only a single line of code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@RobinKastberg @decsny @dleach02 @hakehuang @mmahadevan108 @zephyrbot